Skip to content

samples: net: wifi: Add TLSv1.3 support in wifi example #94313

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nxf58150
Copy link
Contributor

Add TLSv1.3 support in wifi example for RW612 and IW610

Add TLSv1.3 support in wifi example for RW612 and IW610

Signed-off-by: Hui Bai <[email protected]>
Copy link

The following west manifest projects have changed revision in this Pull Request:

Name Old Revision New Revision Diff
hal_nxp zephyrproject-rtos/hal_nxp@2de68b6 zephyrproject-rtos/hal_nxp#585 zephyrproject-rtos/hal_nxp#585/files
hostap zephyrproject-rtos/hostap@c55683c (main) zephyrproject-rtos/hostap#98 zephyrproject-rtos/hostap#98/files

DNM label due to: 2 projects with PR revision

Note: This message is automatically posted and updated by the Manifest GitHub Action.

@github-actions github-actions bot added manifest manifest-hostap manifest-hal_nxp DNM (manifest) This PR should not be merged (controlled by action-manifest) labels Aug 11, 2025
Copy link

@@ -299,6 +299,10 @@ config EAP_ALL
select EAP_TTLS
select EAP_MSCHAPV2
default y

config EAP_TLSV1_3
bool "EAP TLSv1.3 support"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we select MBEDTLS_TLS_SESSION_TICKETS, MBEDTLS_SSL_TLS1_3_KEY_EXCHANGE_MODE_PSK_ENABLED, MBEDTLS_SSL_TLS1_3_KEY_EXCHANGE_MODE_EPHEMERAL_ENABLED, MBEDTLS_SSL_TLS1_3_KEY_EXCHANGE_MODE_PSK_EPHEMERAL_ENABLED here?

@@ -24,6 +24,13 @@ CONFIG_MBEDTLS_PSA_CRYPTO_C=y
CONFIG_MBEDTLS_ENTROPY_C=y
CONFIG_MBEDTLS_SSL_MAX_CONTENT_LEN=8192

# TLSv1.3
CONFIG_EAP_TLSV1_3=n
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if these config are default n, we can remove from here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants